Skip to content

Fixed the Check Links job, updated some links#673

Merged
troyciesco merged 1 commit intomainfrom
fix-links-and-checker
Feb 16, 2026
Merged

Fixed the Check Links job, updated some links#673
troyciesco merged 1 commit intomainfrom
fix-links-and-checker

Conversation

@troyciesco
Copy link
Copy Markdown
Contributor

@troyciesco troyciesco commented Feb 16, 2026

no ref

  • In reviewing some renovate PRs for updating Node versions, I noticed our check links job was failing because it was trying to use Node 24. This is because the workflow is set up to use lts/*, which is 24, but that's not a permitted version in our package.json
  • I'm going to update everything in steps just to make sure nothing gets missed, so the first step is to get check links passing by setting the Node version to 22.
  • In doing so I found some broken links, so this PR fixes them
  • Future PRs like Update dependency node to v24 #632 will update Node to 24, at which point I plan to update check-links.yml as well

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 16, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

Walkthrough

This pull request updates documentation links across multiple specification files and pins the Node.js version in a GitHub Actions workflow. The changes include updating internal documentation anchors from camelCase to kebab-case format (e.g., packagejson to package-json, errorhbs to error-hbs) and modifying a helper link path. The workflow change pins Node.js to version 22 with a TODO note for future updates. No functional logic or control flow is altered.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: fixing the Check Links job by pinning Node.js to version 22, and updating documentation links throughout the codebase.
Description check ✅ Passed The description clearly explains the context and rationale: the workflow was failing due to Node 24 incompatibility, version 22 is being pinned as a temporary fix, and broken links were discovered and fixed in the process.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-links-and-checker

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@troyciesco troyciesco merged commit 57f008a into main Feb 16, 2026
4 checks passed
@troyciesco troyciesco deleted the fix-links-and-checker branch February 16, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@troyciesco